From aca1b6f3911ddc91f7d3861337be2501b264cc67 Mon Sep 17 00:00:00 2001 From: Soeren Sandmann Date: Thu, 30 Oct 2003 17:08:49 +0000 Subject: [PATCH] Make it work even when item 0 is a placeholder. (#125826) add an assertion Thu Oct 30 17:57:53 2003 Soeren Sandmann * gtk/gtktoolbar.c (logical_to_physical): Make it work even when item 0 is a placeholder. (#125826) * tests/testtoolbar.c: add an assertion that gtk_toolbar_get_nth_item (0) != NULL --- ChangeLog | 9 ++++++++- ChangeLog.pre-2-10 | 9 ++++++++- ChangeLog.pre-2-4 | 9 ++++++++- ChangeLog.pre-2-6 | 9 ++++++++- ChangeLog.pre-2-8 | 9 ++++++++- gtk/gtktoolbar.c | 15 +++++++++------ tests/testtoolbar.c | 4 +++- 7 files changed, 52 insertions(+), 12 deletions(-) diff --git a/ChangeLog b/ChangeLog index b8c84fe0d6..f2f3d71e0b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,7 +1,14 @@ +Thu Oct 30 17:57:53 2003 Soeren Sandmann + + * gtk/gtktoolbar.c (logical_to_physical): Make it work even when + item 0 is a placeholder. (#125826) + * tests/testtoolbar.c: add an assertion that + gtk_toolbar_get_nth_item (0) != NULL + Thu Oct 30 17:40:26 2003 Soeren Sandmann * gtk/gtktoolbar.c (logical_to_physical): Make it work for - logical == 0. + logical == 0.(#125826) Thu Oct 30 16:20:58 2003 Soeren Sandmann diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10 index b8c84fe0d6..f2f3d71e0b 100644 --- a/ChangeLog.pre-2-10 +++ b/ChangeLog.pre-2-10 @@ -1,7 +1,14 @@ +Thu Oct 30 17:57:53 2003 Soeren Sandmann + + * gtk/gtktoolbar.c (logical_to_physical): Make it work even when + item 0 is a placeholder. (#125826) + * tests/testtoolbar.c: add an assertion that + gtk_toolbar_get_nth_item (0) != NULL + Thu Oct 30 17:40:26 2003 Soeren Sandmann * gtk/gtktoolbar.c (logical_to_physical): Make it work for - logical == 0. + logical == 0.(#125826) Thu Oct 30 16:20:58 2003 Soeren Sandmann diff --git a/ChangeLog.pre-2-4 b/ChangeLog.pre-2-4 index b8c84fe0d6..f2f3d71e0b 100644 --- a/ChangeLog.pre-2-4 +++ b/ChangeLog.pre-2-4 @@ -1,7 +1,14 @@ +Thu Oct 30 17:57:53 2003 Soeren Sandmann + + * gtk/gtktoolbar.c (logical_to_physical): Make it work even when + item 0 is a placeholder. (#125826) + * tests/testtoolbar.c: add an assertion that + gtk_toolbar_get_nth_item (0) != NULL + Thu Oct 30 17:40:26 2003 Soeren Sandmann * gtk/gtktoolbar.c (logical_to_physical): Make it work for - logical == 0. + logical == 0.(#125826) Thu Oct 30 16:20:58 2003 Soeren Sandmann diff --git a/ChangeLog.pre-2-6 b/ChangeLog.pre-2-6 index b8c84fe0d6..f2f3d71e0b 100644 --- a/ChangeLog.pre-2-6 +++ b/ChangeLog.pre-2-6 @@ -1,7 +1,14 @@ +Thu Oct 30 17:57:53 2003 Soeren Sandmann + + * gtk/gtktoolbar.c (logical_to_physical): Make it work even when + item 0 is a placeholder. (#125826) + * tests/testtoolbar.c: add an assertion that + gtk_toolbar_get_nth_item (0) != NULL + Thu Oct 30 17:40:26 2003 Soeren Sandmann * gtk/gtktoolbar.c (logical_to_physical): Make it work for - logical == 0. + logical == 0.(#125826) Thu Oct 30 16:20:58 2003 Soeren Sandmann diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8 index b8c84fe0d6..f2f3d71e0b 100644 --- a/ChangeLog.pre-2-8 +++ b/ChangeLog.pre-2-8 @@ -1,7 +1,14 @@ +Thu Oct 30 17:57:53 2003 Soeren Sandmann + + * gtk/gtktoolbar.c (logical_to_physical): Make it work even when + item 0 is a placeholder. (#125826) + * tests/testtoolbar.c: add an assertion that + gtk_toolbar_get_nth_item (0) != NULL + Thu Oct 30 17:40:26 2003 Soeren Sandmann * gtk/gtktoolbar.c (logical_to_physical): Make it work for - logical == 0. + logical == 0.(#125826) Thu Oct 30 16:20:58 2003 Soeren Sandmann diff --git a/gtk/gtktoolbar.c b/gtk/gtktoolbar.c index 9095d2ed1e..f6af3f869a 100644 --- a/gtk/gtktoolbar.c +++ b/gtk/gtktoolbar.c @@ -1797,19 +1797,22 @@ logical_to_physical (GtkToolbar *toolbar, gint logical) g_assert (logical >= 0); physical = 0; - for (list = priv->content; list && logical > 0; list = list->next) + for (list = priv->content; list; list = list->next) { ToolbarContent *content = list->data; if (!content->is_placeholder) - logical--; + { + if (logical == 0) + break; + logical--; + } + physical++; - - if (!content->is_placeholder && logical == 0) - break; } - + g_assert (logical == 0); + return physical; } diff --git a/tests/testtoolbar.c b/tests/testtoolbar.c index 04ec128a6f..0388ee4be7 100644 --- a/tests/testtoolbar.c +++ b/tests/testtoolbar.c @@ -625,12 +625,14 @@ main (gint argc, gchar **argv) gtk_toolbar_insert (GTK_TOOLBAR (toolbar), item, -1); gtk_tool_item_set_expand (item, TRUE); gtk_separator_tool_item_set_draw (GTK_SEPARATOR_TOOL_ITEM (item), FALSE); - + g_assert (gtk_toolbar_get_nth_item (toolbar, 0) != 0); + item = gtk_radio_tool_button_new_from_stock (NULL, GTK_STOCK_JUSTIFY_LEFT); group = gtk_radio_tool_button_get_group (GTK_RADIO_TOOL_BUTTON (item)); add_item_to_list (store, item, "Left"); gtk_toolbar_insert (GTK_TOOLBAR (toolbar), item, -1); + item = gtk_radio_tool_button_new_from_stock (group, GTK_STOCK_JUSTIFY_CENTER); #if 0 make_prop_editor (G_OBJECT (item)); -- 2.30.2